Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Миронов Данил #17

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Миронов Данил #17

wants to merge 20 commits into from

Conversation

danmir
Copy link

@danmir danmir commented Mar 30, 2016

  • Хероку
  • "Попробовал бэм на вкус" - но не разобрался с прикручиванием babel для enb -> трансляция прямо в браузере -> надо подождать

onSetMod: { // конструктор для описания реакции на события
'js': {
'inited': function () {
function createNodeFromHtml(html) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут, конечно, адище получился :)

@mokhov
Copy link

mokhov commented Apr 4, 2016

Глобально: молодец! Локально: убери собранные файлы ;) Вечером подниму у себя и начну коментить

@danmir
Copy link
Author

danmir commented Apr 6, 2016

После просмотра лекции Славы Аристова про React.js и Bem, в конце которой он все переписал на bem и выпилил весь реакт - решил не использовать их вместе, а полностью переписать проект. Теперь от бэма только методология.
Отдельно вынес bem версию в ветку и собрал релиз из нее

@mokhov
Copy link

mokhov commented Apr 15, 2016

Приглашаю на менторскую для сдачи

@danmir
Copy link
Author

danmir commented Apr 15, 2016

@mokhov А почему так ? Я же не просрочил сдачу

@danmir
Copy link
Author

danmir commented Apr 26, 2016

@mokhov Как-то тут странно опять с меткой получилось. Я же не просрочил. После менторской доделывал.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants